Skip to content

Conversation

@cauemarcondes
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…, root) (elastic#204977)

## Summary
This PR is a pre-requisite for adding aggregation by process name and by
thread name to the Universal Profiling flamegraph view.

It adds three artificial node types to the flamegraph including color
codes.

As a side-effect, the root node now has its own color code. Previously,
it (accidentally) used the color code of "unknown" type frames.

The PR is backwards compatible, so it doesn't change anything in the UI
when connecting with current Elasticsearch.
As soon as [the PR for
ES](elastic/elasticsearch#119115) is merged, the
new aggregations show up.

(cherry picked from commit bc5d8db)
@rockdaboot
Copy link
Contributor

rockdaboot commented May 5, 2025

@cauemarcondes This PR was required for elastic/elasticsearch#119115, which is 9.1+. Though it won't hurt to get the Kibana changes into 8.19, they are not required or even used.

@cauemarcondes
Copy link
Contributor Author

@rockdaboot we'd like to backport this to 8.19. And this one is required in order to do so.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
profiling 371.6KB 371.7KB +112.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
profiling 18.0KB 18.1KB +155.0B

History

Copy link
Contributor

@rockdaboot rockdaboot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cauemarcondes cauemarcondes merged commit 47f9ce4 into elastic:8.19 May 6, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants